Wire up organization client endpoints into OrganizationsClientsSection by Copilot · Pull Request #324 · panoramicdata/Meraki.Api

Copilot · 2026-04-04T17:28:06Z

IOrganizationsClientOverview and IOrganizationBandwidthUsageHistory interfaces existed but were never wired into MerakiClient or OrganizationsSection. The IOrganizationsClients (search) was wired as a flat interface rather than a proper section.

Changes

New OrganizationsClientsSection — Groups all three client interfaces following the NetworksClientsSection pattern. Uses [RefitPromoteCalls] on the internal IOrganizationsClients for backward compatibility.
OrganizationsSection.Clients — Changed from IOrganizationsClients to OrganizationsClientsSection
MerakiClient wiring — Wires up Clients, BandwidthUsageHistory, and Overview

Existing consumer code is unaffected:

// Still works — promoted via source generator
client.Organizations.Clients.GetOrganizationClientsSearchAsync(orgId, mac);

// Now also available
client.Organizations.Clients.Overview.GetOrganizationClientsOverviewAsync(orgId, t0, t1);
client.Organizations.Clients.BandwidthUsageHistory.GetOrganizationClientsBandwidthUsageHistoryAsync(orgId, t0, t1);

codacy-production · 2026-04-04T17:28:57Z

Not up to standards ⛔

🔴 Issues 1 minor

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results

CodeStyle 1 minor

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

_{TIP This summary will be updated as you push new changes. Give us feedback}

…tory endpoints Create OrganizationsClientsSection to properly group all organization-client interfaces (IOrganizationsClients, IOrganizationsClientOverview, and IOrganizationBandwidthUsageHistory) following the established section pattern. - Add OrganizationsClientsSection.cs with RefitPromoteCalls for backward compat - Update OrganizationsSection.cs Clients property to use new section type - Update MerakiClient.cs to wire up all three client interfaces Agent-Logs-Url: https://github.com/panoramicdata/Meraki.Api/sessions/20dfd578-4d0e-48ec-ab9d-98478d1a1b16 Co-authored-by: danielabbatt <8926756+danielabbatt@users.noreply.github.com>

Copilot

Pull request overview

Wires previously-unreachable organization client-related endpoints into the MerakiClient/OrganizationsSection hierarchy by introducing a dedicated OrganizationsClientsSection, aligning the Organizations “clients” surface with the existing Networks pattern.

Changes:

Added OrganizationsClientsSection to group client search, overview, and bandwidth usage history endpoints.
Updated OrganizationsSection.Clients to expose the new OrganizationsClientsSection.
Updated MerakiClient constructor wiring to initialize the new nested client interfaces.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
Meraki.Api/Sections/General/Organizations/OrganizationsSection.cs	Exposes `Clients` as a section (instead of a flat interface) to match other section hierarchies.
Meraki.Api/Sections/General/Organizations/OrganizationsClientsSection.cs	New section grouping the three organization client interfaces, promoting calls for the legacy search interface.
Meraki.Api/MerakiClient.cs	Wires up the new section’s interfaces (`Clients`, `Overview`, `BandwidthUsageHistory`) via Refit.

Copilot · 2026-04-12T19:28:29Z

 	/// <summary>
-	/// Interface for client search and details.
+	/// Section for client search and details.
 	/// </summary>
-	public IOrganizationsClients Clients { get; internal set; } = null!;
+	public OrganizationsClientsSection Clients { get; internal set; } = new();


Changing OrganizationsSection.Clients from IOrganizationsClients to OrganizationsClientsSection is a public API signature change. [RefitPromoteCalls] preserves method-call syntax, but consumers that reference the property as IOrganizationsClients (e.g., assignments/DI registrations/method params) will no longer compile. If the intent is truly non-breaking, consider keeping a public IOrganizationsClients-typed property (possibly obsolete) or having OrganizationsClientsSection implement IOrganizationsClients so it remains assignable to the interface; otherwise this should be treated/documented as a breaking change (SemVer/changelog).

Copilot · 2026-04-12T19:28:29Z


 	/// <summary>
-	/// Interface for client search and details.
+	/// Section for client search and details.


The XML doc for OrganizationsSection.Clients still describes only “client search and details”, but this section now also exposes Overview and BandwidthUsageHistory endpoints. Update the summary to reflect the full scope to avoid misleading IntelliSense for consumers.

Suggested change

/// Section for client search and details.

/// Section for client search, details, overview, and bandwidth usage history operations.

Copilot · 2026-04-12T19:28:30Z

+			Clients = new()
+			{
+				Clients = RefitFor(Organizations.Clients.Clients),
+				BandwidthUsageHistory = RefitFor(Organizations.Clients.BandwidthUsageHistory),
+				Overview = RefitFor(Organizations.Clients.Overview),
+			},


New public surface area is being wired (Organizations.Clients.Overview and .BandwidthUsageHistory), but there are no corresponding tests exercising these endpoints (only GetOrganizationClientsSearchAsync is covered). Consider adding integration tests alongside Meraki.Api.Test/Organizations/Clients/Tests.cs to ensure these newly exposed interfaces remain correctly wired and callable.

Initial plan

256daca

Copilot AI assigned Copilot and danielabbatt Apr 4, 2026

Copilot started work on behalf of danielabbatt April 4, 2026 17:28 View session

Copilot AI linked an issue Apr 4, 2026 that may be closed by this pull request

Unable to search all organization clients #190

Closed

Copilot AI changed the title ~~[WIP] Add endpoint for searching organization clients~~ Wire up organization client endpoints into OrganizationsClientsSection Apr 4, 2026

Copilot AI requested a review from danielabbatt April 4, 2026 17:38

Copilot finished work on behalf of danielabbatt April 4, 2026 17:38

danielabbatt approved these changes Apr 12, 2026

View reviewed changes

danielabbatt marked this pull request as ready for review April 12, 2026 19:24

Copilot AI review requested due to automatic review settings April 12, 2026 19:24

danielabbatt merged commit f7e1d24 into main Apr 12, 2026
4 of 5 checks passed

Copilot started reviewing on behalf of danielabbatt April 12, 2026 19:25 View session

danielabbatt deleted the copilot/add-search-functionality-for-clients branch April 12, 2026 19:25

Copilot AI reviewed Apr 12, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Wire up organization client endpoints into OrganizationsClientsSection#324

Wire up organization client endpoints into OrganizationsClientsSection#324
danielabbatt merged 2 commits intomainfrom
copilot/add-search-functionality-for-clients

Copilot AI commented Apr 4, 2026 •

edited

Loading

Uh oh!

codacy-production bot commented Apr 4, 2026 •

edited

Loading

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 12, 2026

Uh oh!

Copilot AI Apr 12, 2026

Uh oh!

Copilot AI Apr 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	/// Section for client search and details.
	/// Section for client search, details, overview, and bandwidth usage history operations.

Conversation

Copilot AI commented Apr 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Uh oh!

codacy-production bot commented Apr 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Not up to standards ⛔

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 12, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Apr 4, 2026 •

edited

Loading

codacy-production bot commented Apr 4, 2026 •

edited

Loading